Skip to content

[BUG-451] Allow setting imagePullSecrets for Deployment - #452

Closed
michizubi-SRF wants to merge 3 commits into
jetstack:mainfrom
michizubi-SRF:main
Closed

[BUG-451] Allow setting imagePullSecrets for Deployment#452
michizubi-SRF wants to merge 3 commits into
jetstack:mainfrom
michizubi-SRF:main

Conversation

@michizubi-SRF

@michizubi-SRF michizubi-SRF commented Aug 3, 2026

Copy link
Copy Markdown

This allows setting imagePullSecrets for the Deployment in the Helm Charts

Fixes #451

@davidcollom

Copy link
Copy Markdown
Collaborator

The imagePullSecrets is attached to the service account, not the deployment, this means that we can re-use the imagePullSecrets within the application to authenticate with upstream repositories within version-checker.

You can see this in our helm-unittests here: https://github.com/jetstack/version-checker/blob/main/deploy/charts/version-checker/tests/serviceaccount_test.yaml#L24-L31

Along with the template: https://github.com/jetstack/version-checker/blob/main/deploy/charts/version-checker/templates/serviceaccount.yaml#L12-L14

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] imagePullSecrets missing in template, documented in values.yaml

2 participants